Skip to content

Add Dependency Security Audit workflow #4#5

Open
olatechpro wants to merge 2 commits into
developmentfrom
dependency-security-audit
Open

Add Dependency Security Audit workflow #4#5
olatechpro wants to merge 2 commits into
developmentfrom
dependency-security-audit

Conversation

@olatechpro
Copy link
Copy Markdown

@olatechpro olatechpro commented May 18, 2026

The workflow should run on PRs/commits to detect vulnerable dependencies (Composer packages). Flags packages with known security issues before they reach production. fix #4

The workflow is tested here: https://github.com/publishpress/publishpress-cart/pull/191

@olatechpro olatechpro requested a review from andergmartins May 18, 2026 08:55
@olatechpro
Copy link
Copy Markdown
Author

Hi @andergmartins While testing this workflow on cart, https://github.com/publishpress/publishpress-cart/pull/191,

It's flagging a vulnerability in publishpress/dev-workspace package.

The reported advisory is not coming from plugin runtime code but a dev dependency chain in the lockfile

publishpress/dev-workspace
-> wp-cli/wp-cli-bundle
-> wp-cli/package-command
-> composer/composer 2.9.7

You can see the failure on the link shared above.

So there are two possible directions:

  1. Keep the workflow auditing all dependencies, including dev tooling
    In this case, we need to make a fix in publishpress/dev-workspace, and then we refresh the Cart lockfile with a composer update.

  2. Limit this workflow to production dependencies only
    In this case, I can change the shared workflow from:
    composer audit --locked
    to:
    composer audit --locked --no-dev

That would align the workflow with shipped dependency risk, and avoid failures caused only by CI/dev tooling packages.

Please confirm which policy we want for this workflow:

  • audit all dependencies, including dev
  • audit production dependencies only

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add reusable Dependency Security Audit workflow

1 participant